Skip to content

Limited Library Support#63

Merged
stiefn merged 8 commits intomainfrom
library_handling
May 16, 2025
Merged

Limited Library Support#63
stiefn merged 8 commits intomainfrom
library_handling

Conversation

@ritzdorf
Copy link
Contributor

@ritzdorf ritzdorf commented May 15, 2025

  1. Apparently solidity (since ancient versions) adds a PUSH20 0x00000000000 at the beginning of library deployed bytecode. The zeroes are overwritten in the constructor with the address where the library is deployed. So it behaves very similar to an immutable but is not marked as such. For library bytecode verification we now ignore those bytes.
  2. Workaround in readme on how to work with libraries until Support Compilation with Libraries #60 is resolved
  3. Needed to extract from the AST whether a contract is a library and in the process removed some .clone() operations that presumably led to lots of cloning.

@stiefn stiefn force-pushed the library_handling branch from 74112e6 to 14053a4 Compare May 16, 2025 10:26
@stiefn stiefn merged commit 0393205 into main May 16, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@stiefn stiefn

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ritzdorf @stiefn